-
Notifications
You must be signed in to change notification settings - Fork 25.1k
Update Overview of ASP.NET Core #35847
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
danroth27
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of minor comments. Otherwise, looks great!
Co-authored-by: Daniel Roth <[email protected]>
|
@wadepickett ... Do you want to review? No big deal if you're busy. Dan and Tom have signed off. |
|
@guardrex, Yes, thanks, I have been looking it over, just a little more time please... |
Co-authored-by: Wade Pickett <[email protected]>
wadepickett
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me. Approved. But fix the two typos I pointed out.
|
Thanks for catching those, @wadepickett! 🎸 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR restructures the ASP.NET Core documentation overview by consolidating multiple overview documents into a single, streamlined introduction. The main purpose is to simplify the documentation structure and create a more focused entry point for developers learning ASP.NET Core.
Key changes:
- Replaced the lengthy
introduction-to-aspnet-core.mdwith a conciseoverview.mdthat highlights key features and benefits - Simplified the table of contents structure by removing nested overview items
- Moved sample download instructions from the introduction to the fundamentals section where they're more contextually relevant
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| aspnetcore/overview.md | New streamlined overview document with key ASP.NET Core features and benefits organized by version |
| aspnetcore/introduction-to-aspnet-core.md | Removed the original lengthy introduction document |
| aspnetcore/toc.yml | Simplified TOC structure by removing nested overview items |
| aspnetcore/get-started.md | Enhanced with additional tutorial sections and resources |
| aspnetcore/fundamentals/index.md | Added sample download instructions moved from the introduction |
| aspnetcore/fundamentals/choose-aspnet-framework.md | Removed comparison document between ASP.NET 4.x and Core |
| aspnetcore/includes/benefits.md | Removed shared benefits include file |
| .openpublishing.redirection.json | Added redirects for removed documents |
Comments suppressed due to low confidence (1)
aspnetcore/overview.md:7
- The date '07/28/2025' is in the future and may not be accurate. Documentation dates should reflect when the content was actually created or last updated.
ms.date: 07/28/2025
|
I ran Copilot on the last version of this PR, not this one by mistake. I'm still getting used to doing it, and I do like it! 👍 |
Fixes #35739
Fixes #35823
Internal previews